Skip to content

Conversation

tjuhaszrh
Copy link
Contributor

@tjuhaszrh tjuhaszrh commented Aug 27, 2025

Description

Installation method in Fedora/CentOs/RHEL space has changed in the last few years, moving away from using modules to using traditional rpms for all maintained stream versions.

Quick search on google suggests that this created at-least some confusion with the conflicting content of the page modified in this PR.

Related Issues

No related issues

Check List

  • I have read the Contributing Guidelines and made commit messages that follow the guideline.
  • I have run pnpm format to ensure the code follows the style guide.
  • I have run pnpm test to check if all tests are passing.
  • I have run pnpm build to check if the website builds without errors.
  • [] I've covered new added functionality with unit tests if necessary.

@tjuhaszrh tjuhaszrh requested a review from a team as a code owner August 27, 2025 13:48
Copy link

vercel bot commented Aug 27, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Updated (UTC)
nodejs-org Ready Ready Preview Sep 1, 2025 7:32am

Installation method in Fedora/CentOs/RHEL space has changed in the last few years, moving away from using modules to using traditional rpms for all maintained stream versions.
@tjuhaszrh tjuhaszrh force-pushed the update-install-method branch from 89ecbbe to f036d6a Compare August 27, 2025 13:56
Copy link
Member

@AugustinMauroy AugustinMauroy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@araujogui araujogui requested a review from Copilot August 27, 2025 22:43
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR updates the installation instructions for Node.js on Fedora, CentOS, and RHEL systems to reflect current package management practices. The documentation has been modernized to show the transition from module-based installation to traditional RPM packages for maintained distributions.

  • Updated installation method from modules to traditional dnf package installation for current Fedora and RHEL 10
  • Added instructions for installing specific Node.js versions using stream-specific packages
  • Clarified that module-based installation is only for older CentOS/RHEL releases

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

tjuhaszrh and others added 2 commits August 28, 2025 10:52
Co-authored-by: Copilot <[email protected]>
Signed-off-by: tjuhaszrh <[email protected]>
Co-authored-by: Copilot <[email protected]>
Signed-off-by: tjuhaszrh <[email protected]>
tjuhaszrh and others added 2 commits August 29, 2025 09:28
Co-authored-by: Michael Esteban <[email protected]>
Signed-off-by: tjuhaszrh <[email protected]>
Co-authored-by: Michael Esteban <[email protected]>
Signed-off-by: tjuhaszrh <[email protected]>
Co-authored-by: Michael Esteban <[email protected]>
Signed-off-by: tjuhaszrh <[email protected]>
Copy link

codecov bot commented Sep 1, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 76.60%. Comparing base (a5ea254) to head (45a222a).
⚠️ Report is 8 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #8114      +/-   ##
==========================================
+ Coverage   75.89%   76.60%   +0.70%     
==========================================
  Files         113      115       +2     
  Lines        9459     9595     +136     
  Branches      307      321      +14     
==========================================
+ Hits         7179     7350     +171     
+ Misses       2279     2244      -35     
  Partials        1        1              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Member

@mikeesto mikeesto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @tjuhaszrh !

@mikeesto mikeesto added this pull request to the merge queue Sep 1, 2025
Merged via the queue into nodejs:main with commit 9f16142 Sep 1, 2025
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants